-
Notifications
You must be signed in to change notification settings - Fork 489
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add more fields in DiffSchema for exchanging partition #883
Conversation
Codecov Report
@@ Coverage Diff @@
## master #883 +/- ##
==========================================
- Coverage 78.32% 78.29% -0.03%
==========================================
Files 40 40
Lines 14708 14700 -8
==========================================
- Hits 11520 11510 -10
- Misses 2505 2507 +2
Partials 683 683 |
Co-authored-by: Lynn <zimu_xia@126.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PTAL @tiancaiamao @kennytm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides, maybe we need to add a field to BinlogInfo
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides, maybe we need to add a field to BinlogInfo.
Confirmed with classmates of drainer and CDC, no new fields need to be added at present.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM |
Co-authored-by: Lynn <zimu_xia@126.com> Co-authored-by: kennytm <kennytm@gmail.com> Co-authored-by: tiancaiamao <tiancaiamao@gmail.com>
What problem does this PR solve?
While exchanging partition affects 2 tables, we need to expand
DiffSchema
. Related PR in TIDB #17149What is changed and how it works?
Check List
Tests
Side effects
Release note